-
Notifications
You must be signed in to change notification settings - Fork 271
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for V2 #137
Add support for V2 #137
Conversation
Hi, |
There is a preliminary user guide that has a description of the protocol: Some users have been sent sample/demo units. Wide availability of the hardware will begin after Chinese New Year. |
Hi @nanovna I have a PR open that may need adjusting if yours is merged first. Could I ask your help testing my PR with the v2 nanoVNA? It basically adds a "pause stimulus" serial command after scan/sweep. You should only need to add a Let me know if it works on the v2 nanoVNA, thanks! |
Hi, I'll be doing a rebase later after your patch is merged, and then I can fix any issues that come up. |
I don't know when Rune gets time for working on NanoVNA-Saver again (and when he get's his V2). And as Rune noted somewhere, he isn't working on master branch, so he likes to get MRs to the Development branch. Regards P.s.: There seems to be some problems with S21 measurement with that patch (mentioned in groups.io mailing list. But I've not got the time to check and conform this issue. |
I have merged and adjusted the code for the current Development branch and made it available as master in my repository https://github.com/zarath/nanovna-saver.git But not fixed that mentioned s21 issue |
included in v0.3.0 |
This patch adds support for V2 (S-A-A-2) hardware.
I have only tested this on Linux (debian), consider testing more platforms and more V1 device/firmware variants to make sure it doesn't break anything before merging.